Skip to content

Enable rtc on rt700 #93006

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Jul 28, 2025
Merged

Enable rtc on rt700 #93006

merged 1 commit into from
Jul 28, 2025
+166 −40

Conversation

Holt-Sun
Copy link
Contributor

Updated rtc driver to support NXP RT700.

  1. Update nxp irtc driver to fix issue in init and alarm get function.
  2. Update RTC device tree binding to support "share-counter".
  3. Update RT700 dtsi to support rtc0 for cpu0 and rtc1 for cpu1.
  4. Update readme.

Enable rt700 RTC unittest.

@github-actions github-actions bot added area: Samples Samples platform: NXP Drivers NXP Semiconductors, drivers area: RTC Real Time Clock platform: NXP NXP labels Jul 11, 2025
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch from c1b3d9c to 55a2226 Compare July 11, 2025 09:42
bjarki-andreasen
bjarki-andreasen requested changes Jul 11, 2025
View reviewed changes
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch 2 times, most recently from 620191f to 9ff07ae Compare July 16, 2025 03:17
zejiang0jason
zejiang0jason reviewed Jul 17, 2025
View reviewed changes
zejiang0jason
zejiang0jason reviewed Jul 17, 2025
View reviewed changes
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch 5 times, most recently from 1bd320d to 1a9a139 Compare July 21, 2025 07:55
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch from 1a9a139 to 3bc1703 Compare July 24, 2025 02:45
bjarki-andreasen
bjarki-andreasen reviewed Jul 24, 2025
View reviewed changes
Copy link
Contributor

@bjarki-andreasen bjarki-andreasen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Changes look fine, next are the CI failures :)

@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch from 3bc1703 to f3c137f Compare July 25, 2025 03:19
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch from f3c137f to eb13915 Compare July 25, 2025 03:28
@Holt-Sun
drivers: irtc: Updated rtc driver to support NXP RT700 device.
fc10047 
1. Update nxp irtc driver to fix issue in init and alarm function.
2. Update RTC device tree binding to support "share-counter".
3. Update RT700 dtsi to support rtc0 for cpu0 and rtc1 for cpu1.
4. Update readme.
5. Update unit test project conf for RT700.

Signed-off-by: Holt Sun <[email protected]>
@Holt-Sun Holt-Sun force-pushed the enable-rtc-on-rt700 branch from eb13915 to fc10047 Compare July 25, 2025 05:27
@sonarqubecloud SonarQubeCloud
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@Holt-Sun
Copy link
Contributor Author

Changes look fine, next are the CI failures :)
Hi @bjarki-andreasen , from https://github.com/zephyrproject-rtos/zephyr/pull/93006/checks, there are no CI failures. Thanks.

@cfriedt cfriedt merged commit 4618e86 into zephyrproject-rtos:main Jul 28, 2025
28 checks passed
@github-actions GitHub Actions
Copy link

Hi @Holt-Sun!
Congratulations on getting your very first Zephyr pull request merged 🎉🥳. This is a fantastic achievement, and we're thrilled to have you as part of our community!

To celebrate this milestone and showcase your contribution, we'd love to award you the Zephyr Technical Contributor badge. If you're interested, please claim your badge by filling out this form: Claim Your Zephyr Badge.

Thank you for your valuable input, and we look forward to seeing more of your contributions in the future! 🪁

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dleach02 dleach02 dleach02 approved these changes

@zejiang0jason zejiang0jason zejiang0jason approved these changes

@bjarki-andreasen bjarki-andreasen bjarki-andreasen approved these changes

@butok butok Awaiting requested review from butok

@dbaluta dbaluta Awaiting requested review from dbaluta

@decsny decsny Awaiting requested review from decsny

@DerekSnell DerekSnell Awaiting requested review from DerekSnell

@EmilioCBen EmilioCBen Awaiting requested review from EmilioCBen

@kartben kartben Awaiting requested review from kartben

@manuargue manuargue Awaiting requested review from manuargue

@mmahadevan108 mmahadevan108 Awaiting requested review from mmahadevan108

@nashif nashif Awaiting requested review from nashif

@Raymond0225 Raymond0225 Awaiting requested review from Raymond0225

Assignees

@bjarki-andreasen bjarki-andreasen

Labels
area: RTC Real Time Clock area: Samples Samples platform: NXP Drivers NXP Semiconductors, drivers platform: NXP NXP
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@Holt-Sun @dleach02 @zejiang0jason @bjarki-andreasen @cfriedt